Skip to content

1342 allow to set permissions to exclude to empty array overriding defaults#1347

Merged
waldekmastykarz merged 12 commits into
dotnet:mainfrom
bartizan:1342_allow-to-set-permissionsToExclude-to-empty-array-overriding-defaults
Aug 5, 2025
Merged

1342 allow to set permissions to exclude to empty array overriding defaults#1347
waldekmastykarz merged 12 commits into
dotnet:mainfrom
bartizan:1342_allow-to-set-permissionsToExclude-to-empty-array-overriding-defaults

Conversation

@bartizan

Copy link
Copy Markdown
Contributor

@bartizan bartizan marked this pull request as ready for review July 25, 2025 15:11
@bartizan bartizan requested a review from a team as a code owner July 25, 2025 15:11

@waldekmastykarz waldekmastykarz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Let's do a few small adjustments before we merge

Comment thread DevProxy.Abstractions/Plugins/BasePlugin.cs Outdated
Comment thread DevProxy.Abstractions/Plugins/BasePlugin.cs Outdated
Comment thread DevProxy.Plugins/Reporting/GraphMinimalPermissionsGuidancePlugin.cs Outdated
Comment thread DevProxy.Abstractions/Plugins/BasePlugin.cs Outdated
@waldekmastykarz waldekmastykarz merged commit c9f3779 into dotnet:main Aug 5, 2025
4 checks passed
@waldekmastykarz waldekmastykarz added the pr-bugfix Fixes a bug label Aug 5, 2025
@bartizan bartizan deleted the 1342_allow-to-set-permissionsToExclude-to-empty-array-overriding-defaults branch August 5, 2025 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to override default excluded permissions with an empty array in GraphMinimalPermissionsGuidancePlugin

2 participants